Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(createProductCategoriesWorkflow): add productCategoriesCreated hook #10383

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

workylus
Copy link

@workylus workylus commented Dec 1, 2024

What

This commit introduces the productCategoriesCreated hook to the createProductCategoriesWorkflow. It also fixes inconsistent variable names within the workflow.

Why

The hook was missing and necessary to improve the workflow's extensibility.

  • Variable names like createdProducts were misleading and did not match the context. They were updated to createdProductCategories for clarity.

How

Added the productCategoriesCreated hook and updated related logic within the workflow.

  • Renamed variables from createdProducts to createdProductCategories to reflect their actual purpose.

Fixes: #10230

### What
This commit introduces the `productCategoriesCreated` hook to the `createProductCategoriesWorkflow`.
It also fixes inconsistent variable names within the workflow.

### Why
The hook was missing and necessary to improve the workflow's extensibility.
- Variable names like `createdProducts` were misleading and did not match the context. They were updated to `createdProductCategories` for clarity.

### How
Added the `productCategoriesCreated` hook and updated related logic within the workflow.
- Renamed variables from `createdProducts` to `createdProductCategories` to reflect their actual purpose.

Fixes: medusajs#10230
Copy link

changeset-bot bot commented Dec 1, 2024

⚠️ No Changeset found

Latest commit: 7242653

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Dec 1, 2024

@workylus is attempting to deploy a commit to the medusajs Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Missing Hooks for createProductCategoriesWorkflow and createInventoryItemsWorkflow
1 participant